Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: call aws secretsmanager #1704

Closed
wants to merge 1 commit into from

Conversation

katebygrace
Copy link
Contributor

JIRA:CLOUDSEC-12


source secrets-manager.sh analytics-secure/warehouse-transforms/profiles DBT_PASSWORD

dbt deps --profiles-dir . --profile $DBT_PROFILE --target $DBT_TARGET
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think using . for --profiles-dir would work, as the current working directory would be of the warehouse-transforms repo clone on jenkins.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh interesting - should i add the files below there?

@katebygrace katebygrace force-pushed the katebygrace/dbt-secretsmanager branch 3 times, most recently from c9e9f58 to 588364e Compare November 28, 2023 15:38
@katebygrace katebygrace force-pushed the katebygrace/dbt-secretsmanager branch from 588364e to b241c77 Compare November 29, 2023 15:36
dbt deps --profiles-dir $WORKSPACE/analytics-secure/warehouse-transforms/ --profile $DBT_PROFILE --target $DBT_TARGET
source secrets-manager.sh analytics-secure/warehouse-transforms/profiles DBT_PASSWORD

dbt deps --profiles-dir $WORKSPACE/dataeng/profiles --profile $DBT_PROFILE --target $DBT_TARGET
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the workspace is a jenkins provided environment variable that points to the jenkins job workspace.
jenkins-job-dsl repo. does not get cloned unless we specify it in the DSL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants